[feat/#12] 7주차 과제 구현 #13
Conversation
kangyein9892
requested review from
jm991014,
boiledEgg-s,
jihyunniiii and
serioushyeon
| data class SnackBar(@StringRes val message: Int): MySideEffect() | ||
| data class SnackBarText(val message: String): MySideEffect() | ||
| data object Logout: MySideEffect() | ||
| data object NavigateToSignIn: MySideEffect() |
| val intent = _intent.asSharedFlow() | ||
| override suspend fun handleEvent(event: MyEvent) { | ||
| when(event) { | ||
| MyEvent.OnLogoutButtonClick -> { |
There was a problem hiding this comment.
getMyHobby는 data class 상태(프로퍼티)를 가지며, is로 타입 검사를 통해 특정 클래스로 매칭이 필요합니다 ! onLoginButtonClick은 data object이기 때문에 상태가 없는 단일 객체로, 고유 값으로 비교가 가능합니다!
| fun onLogOutButtonClick() = viewModelScope.launch { | ||
| _intent.emit(MySideEffect.Logout) | ||
| private fun navigateToSignIn() = viewModelScope.launch { | ||
| delay(100) |
There was a problem hiding this comment.
화면 이동 되면서 스낵바가 안 떠서 delay를 줬는데 좋은 방법은 아닌 것 같아요... 다른 방법이 있을까요 ? ㅠㅠ
제가 상태관리를 잘못했나봐요...
라고 pr 글에서 말했던.... 이유입니다.......
| is SignInSideEffect.SnackBarText -> snackBarHostState.showSnackbar(sideEffect.message) | ||
| SignInSideEffect.NavigateToSignUp -> { | ||
| navigateToSignUp() |
There was a problem hiding this comment.
위에도 있는 질문인데 is의 유무로 인해 어떤 차이가 있는 건가요!?
There was a problem hiding this comment.
#13 (comment)
요기 달았어요!
코드 써보시면 알겠지만 is가 없다면 빨간줄이 뜹니다~
| fun onLogOutButtonClick() = viewModelScope.launch { | ||
| _intent.emit(MySideEffect.Logout) | ||
| private fun navigateToSignIn() = viewModelScope.launch { | ||
| delay(100) |
| fun onLogOutButtonClick() = viewModelScope.launch { | ||
| _intent.emit(MySideEffect.Logout) | ||
| private fun navigateToSignIn() = viewModelScope.launch { | ||
| delay(100) |
| } | ||
|
|
||
| fun clearUserPreference() = viewModelScope.launch { | ||
| userRepository.clearUserPreference() | ||
| } | ||
|
|
||
| suspend fun handleMyIntentError(message: String) { | ||
| _intent.emit(MySideEffect.SnackBarText(message)) | ||
| fun handleMyIntentError(message: String) = viewModelScope.launch { |
There was a problem hiding this comment.
UI단에서 이미 코루틴을 사용하는 것으로 보이는데, suspend를 지우고 viewModelScope를 넣으신 이유가 있나요?
There was a problem hiding this comment.
엇 그러게요.... 이부분은 제가 중복을 생각못하고 만든것 같네요 !! 수정하겠습니다 ㅎㅎ
| is SignInSideEffect.SnackBarText -> snackBarHostState.showSnackbar(sideEffect.message) | ||
| SignInSideEffect.NavigateToSignUp -> { | ||
| navigateToSignUp() |
| _state.update { | ||
| it.copy(hobby = result.hobby) | ||
| } | ||
| setState { copy(hobby = result.hobby) } |
There was a problem hiding this comment.
석준 오빠 코드리뷰 읽어보시면 좋을 듯 합니다! 이해 안 되면 저 디코로 호출하3
Related issue 🛠
Work Description ✏️
Screenshot 📸
화면 바뀐 부분은 없습니다 !
To Reviewers 📢